Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove terminated connections from the pool. #51

Merged
merged 1 commit into from
Dec 27, 2013
Merged

Remove terminated connections from the pool. #51

merged 1 commit into from
Dec 27, 2013

Conversation

sontek
Copy link
Contributor

@sontek sontek commented Dec 16, 2013

In pymssql, if you terminate a long running query manually
it will give you a connection reset by peer message, but this
connection remains in the pool and will be re-used.

In pymssql, if you terminate a long running query manually
it will give you a connection reset by peer message, but this
connection remains in the pool and will be re-used.
"Connection is closed"
'Error 10054',
'Not connected to any MS SQL server',
'Connection is closed'
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The quoting was inconsistent in the file, so I cleaned it up, but can remove that if you don't like it.

@zzzeek
Copy link
Owner

zzzeek commented Dec 16, 2013

ok, got a few pullreqs I need to get to, should be fine

zzzeek added a commit that referenced this pull request Dec 27, 2013
Remove terminated connections from the pool.
@zzzeek zzzeek merged commit 69876ad into zzzeek:master Dec 27, 2013
@zzzeek
Copy link
Owner

zzzeek commented Dec 27, 2013

changelog: c837dc4 cherry-pick in 0.8 4211207 351d0b6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants